refactor: ci/cd 오류 수정 flyway 버전 - #1448

app-main/src/test/java/net/causw/app/main/domain/notification/notification/service/listener/PostLikeMilestonePushListenerTest.java (1)

54-64: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

비동기와 새 트랜잭션 계약도 검증하세요.

현재 테스트는 TransactionPhase.AFTER_COMMIT만 검증합니다. @Async("asyncExecutor") 또는 @Transactional(propagation = Propagation.REQUIRES_NEW)가 제거되어도 테스트가 통과합니다.

두 애너테이션과 설정값을 함께 검증하세요.

검증 코드 예시

+import org.springframework.scheduling.annotation.Async;
+import org.springframework.transaction.annotation.Propagation;
+import org.springframework.transaction.annotation.Transactional;
+
  TransactionalEventListener annotation = handleMethod.getAnnotation(TransactionalEventListener.class);
+ Async asyncAnnotation = handleMethod.getAnnotation(Async.class);
+ Transactional transactionalAnnotation = handleMethod.getAnnotation(Transactional.class);

  assertThat(annotation).isNotNull();
  assertThat(annotation.phase()).isEqualTo(TransactionPhase.AFTER_COMMIT);
+ assertThat(asyncAnnotation).isNotNull();
+ assertThat(asyncAnnotation.value()).isEqualTo("asyncExecutor");
+ assertThat(transactionalAnnotation).isNotNull();
+ assertThat(transactionalAnnotation.propagation()).isEqualTo(Propagation.REQUIRES_NEW);

As per path instructions, "테스트 케이스가 충분한지 확인합니다 (성공/실패/경계 케이스)."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@app-main/src/test/java/net/causw/app/main/domain/notification/notification/service/listener/PostLikeMilestonePushListenerTest.java`
around lines 54 - 64, Update handle_shouldRunAfterCommit in
PostLikeMilestonePushListenerTest to also retrieve and assert the listener
method’s `@Async` annotation value is "asyncExecutor" and its `@Transactional`
annotation propagation is Propagation.REQUIRES_NEW, while preserving the
existing AFTER_COMMIT assertion.

Source: Path instructions

app-main/src/main/java/net/causw/app/main/domain/notification/notification/service/listener/PostLikeMilestoneReachedListener.java (1)

21-26: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick win

비동기 리스너에서 예외를 로깅해 주세요.

handle은 @Async의 void 메서드입니다. 예외가 발생하면 호출 스택으로 전파되지 않고 AsyncUncaughtExceptionHandler로 넘어갑니다. 전역 핸들러가 없으면 실패 원인이 남지 않습니다. 좋아요 트랜잭션은 이미 커밋된 상태이므로 사용자 요청에도 오류가 드러나지 않습니다.

SLF4J로 실패를 기록해 주세요. 이력 ID와 게시글 ID를 남기면 재처리 시 추적이 쉬워집니다.

♻️ 제안 변경

+import lombok.extern.slf4j.Slf4j;
+
 `@Component`
+@Slf4j
 `@RequiredArgsConstructor`
 public class PostLikeMilestoneReachedListener {
 
 	private final PostLikeMilestoneAchievementRecorder achievementRecorder;
 	private final PostLikeMilestoneNotificationProcessor notificationProcessor;
 
 	`@Async`("asyncExecutor")
 	`@TransactionalEventListener`(phase = TransactionPhase.AFTER_COMMIT)
 	public void handle(PostLikeMilestoneReachedEvent event) {
-		achievementRecorder.record(event)
-			.ifPresent(notificationProcessor::process);
+		try {
+			achievementRecorder.record(event)
+				.ifPresent(notificationProcessor::process);
+		} catch (Exception exception) {
+			log.error("게시글 좋아요 마일스톤 알림 처리 실패. postId={}, milestoneCount={}",
+				event.postId(), event.milestoneCount(), exception);
+		}
+	}

민감 정보를 남기지 않도록 likerId는 로그에서 제외했습니다.

As per path instructions: "로깅은 SLF4J를 사용하고 System.out.println은 절대 사용하지 않습니다."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@app-main/src/main/java/net/causw/app/main/domain/notification/notification/service/listener/PostLikeMilestoneReachedListener.java`
around lines 21 - 26, Update PostLikeMilestoneReachedListener.handle to catch
exceptions from the asynchronous
achievementRecorder.record/notificationProcessor flow and log them through an
SLF4J logger. Include the history ID and post ID in the failure log for
traceability, exclude likerId, and preserve the existing processing behavior.

Source: Path instructions

app-main/src/main/java/net/causw/app/main/domain/notification/notification/service/implementation/PostLikeMilestoneNotificationProcessor.java (1)

35-40: 🩺 Stability & Availability | 🔵 Trivial | 🏗️ Heavy lift

PENDING 상태로 남은 이력의 복구 경로를 마련해 주세요(운영 조언).

process가 중간에 예외로 실패하면 이력은 PENDING으로 남습니다. 현재 흐름에는 재시도 주체가 없어서 해당 마일스톤 알림은 영구 누락됩니다.

PENDING 상태와 생성 시각을 기준으로 재처리하는 배치 잡을 추가하면 좋습니다. process는 이미 상태 검사로 멱등성을 확보하므로 배치 재실행에 안전합니다.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@app-main/src/main/java/net/causw/app/main/domain/notification/notification/service/implementation/PostLikeMilestoneNotificationProcessor.java`
around lines 35 - 40, PostLikeMilestoneNotificationProcessor의 process에서 예외로
PENDING에 남은 이력을 복구할 재처리 배치를 추가하세요. PENDING 상태이면서 생성 시각이 기준 시간을 지난
PostLikeMilestoneAchievement를 조회해 각 achievementId로 process를 호출하고, 기존 process의 상태
검사와 멱등성을 그대로 활용하세요.

app-main/src/test/java/net/causw/app/main/domain/notification/notification/service/implementation/PostLikeMilestoneNotificationProcessorTest.java (1)

99-104: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

저장되는 알림 본문도 검증하면 좋겠습니다.

현재 save(any(Notification.class))만 검증하므로 서비스 알림의 제목·본문 회귀를 잡지 못합니다. 푸시 문구는 이벤트로 검증되지만 서비스 알림 문구는 검증 공백입니다.

ArgumentCaptor로 저장된 Notification을 포착해 문구를 확인해 주세요.

♻️ 제안 변경

 			InOrder inOrder = inOrder(notificationWriter, achievementWriter, eventPublisher);
-			inOrder.verify(notificationWriter).save(any(Notification.class));
+			ArgumentCaptor<Notification> notificationCaptor = ArgumentCaptor.forClass(Notification.class);
+			inOrder.verify(notificationWriter).save(notificationCaptor.capture());
+			assertThat(notificationCaptor.getValue().getTitle())
+				.isEqualTo(String.format("게시물이 좋아요 %d개를 달성했습니다!", milestoneCount));
 			inOrder.verify(notificationWriter).saveLog(postWriter, notification);

org.mockito.ArgumentCaptor 임포트가 필요합니다. Notification의 실제 접근자 이름에 맞춰 조정해 주세요.

As per path instructions: "테스트 케이스가 충분한지 확인합니다 (성공/실패/경계 케이스)."

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@app-main/src/test/java/net/causw/app/main/domain/notification/notification/service/implementation/PostLikeMilestoneNotificationProcessorTest.java`
around lines 99 - 104, Update the notification verification in
PostLikeMilestoneNotificationProcessorTest to capture the Notification passed to
notificationWriter.save using ArgumentCaptor, then assert its title and body
match the expected service-notification text. Preserve the existing InOrder
verification and event assertions, and add coverage for the relevant
success/failure/boundary cases only if needed to validate the message behavior.

Source: Path instructions

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes